Skip to content

Comments

feat(niri): niri depends on socket#4834

Merged
Alexays merged 2 commits intoAlexays:masterfrom
khaneliman:niri
Feb 23, 2026
Merged

feat(niri): niri depends on socket#4834
Alexays merged 2 commits intoAlexays:masterfrom
khaneliman:niri

Conversation

@khaneliman
Copy link
Contributor

@khaneliman khaneliman commented Feb 9, 2026

Don't attempt to use niri modules when socket connection fails. Prevents rendering modules when running another compositor. In same concept as previous Hyprland change 4295faa

Trying niri out and noticed this issue we previously had with Hyprland workspaces. When enabling multiple compositors in a same config, we should only render what's relevant / usable.
image

AFTER:

[2026-02-09 08:58:34.824] [warning] module niri/workspaces: Disabling module "niri/workspaces", Niri IPC: NIRI_SOCKET was not set! (Is Niri running?)
[2026-02-09 08:58:34.841] [warning] module sway/workspaces: Disabling module "sway/workspaces", Socket path is empty
[2026-02-09 08:58:34.843] [warning] module niri/window: Disabling module "niri/window", Niri IPC: NIRI_SOCKET was not set! (Is Niri running?)
[2026-02-09 08:58:34.858] [warning] module sway/window: Disabling module "sway/window", Socket path is empty

Properly disable the modules to not render blank spaces.

khaneliman added a commit to khaneliman/khanelinix that referenced this pull request Feb 9, 2026
Prevent invalid niri workspace module from rendering box. Remove after
upstreamed in Alexays/Waybar#4834
khaneliman added a commit to khaneliman/khanelinix that referenced this pull request Feb 9, 2026
Prevent invalid niri workspace module from rendering box. Remove after
upstreamed in Alexays/Waybar#4834
@khaneliman
Copy link
Contributor Author

@Alexays if you get a chance. just a QOL thing for people using multiple different compositors on the same machine

Don't attempt to use niri modules when socket connection fails. Prevents
rendering modules when running another compositor. In same concept as
previous Hyprland change
Alexays@4295faa

Signed-off-by: Austin Horstman <khaneliman12@gmail.com>
Move constructor from hpp to cpp to align with other modules

Signed-off-by: Austin Horstman <khaneliman12@gmail.com>
@khaneliman
Copy link
Contributor Author

@Alexays friendly bump

@Alexays Alexays merged commit 730e558 into Alexays:master Feb 23, 2026
9 checks passed
@Alexays
Copy link
Owner

Alexays commented Feb 23, 2026

Thanks!
Sorry for the delay

@khaneliman khaneliman deleted the niri branch February 24, 2026 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants